Remove an unused function

Remove a self-assignment

clang doesn't like those.

Fix a doc comment typo

Updated Turkish translation

Updated Portuguese translation

Formatting fixes

HighContrast: Fix undershoot in sidebars

This was just not implemented, and the sidebars solid white
background color was inherited by undershoot rectangle.

https://bugzilla.gnome.org/show_bug.cgi?id=753391

listbox: Avoid using show_all on rows

Don't use gtk_widget_show_all() on row widgets because that would
unconditionally show all of its children. This might be unwanted in case
the row implementation wants to keep some of its children hidden.

This commit changes it to use show() instead of show_all() and relies on
the row widget to control the visibility of its children itself as
appropriate.

https://bugzilla.gnome.org/show_bug.cgi?id=753392

Updated Lithuanian translation

Update Catalan translation

quartz: Implement CSD drag resize for all edges/corners

Manual resizing for CSD windows only worked with the bottom
right corner. This extends it to work for all corners and edges.

https://bugzilla.gnome.org/show_bug.cgi?id=753329

HighContrast: Improve GtkCalendar drawing

Make days gray that are not in the current month.

https://bugzilla.gnome.org/show_bug.cgi?id=753357

file chooser button: Fix some refcounting confusion

GtkFileSystem has a complicated way to handle cancellables.
You keep the cancellable pointer that is returned by
_gtk_file_system_get_info and similar methods so that you can
cancel the operation, but you do not own a reference to it.
The only place where it is ok to unref a cancellable is in
your callback, which gets handed a cancellable that you need
to unref at the end. You are expected to compare it to the
pointer you stashed away to find out if the operation has
already been superseded by a newer call, in which case you
disregard the results.

GtkFileChooserButton was following these rules for most of
the cancellables it keeps around, but it was sometimes unreffing
the cancellables that are stored in the model, which could lead
to refcount confusion and crashes. This commit makes it follow
the rules for that case too, which fixes the crash in the bug
below, and does not show up any leaks in valgrind under light
testing.

https://bugzilla.gnome.org/show_bug.cgi?id=737804

gtk/gtkcsstypesprivate.h: Fix Build

The recent change to the enum declaration for GtkCssChange actually
relied on compiler-dependent behavior, which also breaks the build on
some non-GCC compilers, such as Visual Studio.  As noted in the
G_STATIC_ASSERT line just beneath this declaration, we need to change
this enum declaration to #define's, in order to fix the build in such
situations.

https://bugzilla.gnome.org/show_bug.cgi?id=752814

3.17.6

Fix a doc comment

Documentation fixes

Updates

placesview: Fix wrong property type when creating a GtkLabel

The type of xalign property of GtkLabel is float, not int. Using wrong type
crashes GtkFileChooser on x86_64 when compiling with clang.

https://bugzilla.gnome.org/show_bug.cgi?id=753284

GtkScrolledWindow: Don't handle key event when can't scroll

Don't return that a key event was handled when the corresponding
scrollbar can not scroll.

https://bugzilla.gnome.org/show_bug.cgi?id=753256

Adwaita: calendar button fix

- provide insensitive button state for GtkCalendar buttons

https://bugzilla.gnome.org/show_bug.cgi?id=753230

color chooser: Use a popover for the context menu

It works just as well here as it does in the file chooser, and
this lets us unify the right-click and long-press behavior a bit.
We used to switch directly to the editor on long-press, now we
can show the popover, just as we do on right-click.

wayland: Don't assume drag context has a source window when finalizing

Only a drag context which was created with 'drag_begin' will be
guaranteed to have a source window at all times. Thus, in finalize we
cannot assume we can retrieve a GdkDisplay from the source_window
pointer since it may be NULL. Though, the display is only needed for
contexts created via 'drag_begin' thus we can retrieve it after
checking that is the case.

https://bugzilla.gnome.org/show_bug.cgi?id=749339

gtkwindow: bail out on WM operations in the presence of grabs

These operations will require a grab on the WM side, so we can spare the
attempt from the WM to take a grab when we're certain it won't suceed.

https://bugzilla.gnome.org/show_bug.cgi?id=752327

gtkmain: Don't process WM events if there's a grab on another toplevel

This is implicitly done for us in the case of grabs on windows from other
groups, but we must perform this check explicitly for grabs with
owner_events=True on windows from the same group, in that case the window
would handle the events as if there was no grab.

https://bugzilla.gnome.org/show_bug.cgi?id=752327

inspector: Fix a typo

Blocking a signal should be paired with unblocking.
Spotted by Krzesimir Nowak

GtkButton: Pick up images that are set early

We were failing to construct the child if an image was set
before constructed (e.g. in the init() function of a button
subclass).

https://bugzilla.gnome.org/show_bug.cgi?id=753048

HighContrast: Avoid resizing treeview headers

https://bugzilla.gnome.org/show_bug.cgi?id=753060

Whitespace fixes

wayland: Ensure modal hint gets set on map

We need to be mapped to have a gtk_surface and thus be able to do
requests on it so we need to save the modal hint and apply it when we
get mapped so that code that sets the hint before showing a window
doesn't get ignored.

https://bugzilla.gnome.org/show_bug.cgi?id=753138

wayland: Invalidate our gtk_surface when we're unmapped

Otherwise if we get mapped again we'll try to use an invalid
gtk_surface and the compositor will disconnect us.

https://bugzilla.gnome.org/show_bug.cgi?id=753138

GtkLabel: Show erroneous text if parsing markup failed

https://bugzilla.gnome.org/show_bug.cgi?id=752920

file chooser: Don't leave out icons by accident

We only load thumbnails when we find that the row is in the visible
range of the treeview. It seems that animated scrolling makes it so
that the bottommost row stays out of the visible range until it is
too late. To work around this, extend the range by one row in each
direction.

http://bugzilla.gnome.org/show_bug.cgi?id=753142

Adwaita: Remove non-working selector

This was suppsed to be ".list-row.button", I'd imagine, but the missing
dot meant it never worked in the first place. Just clean it up.

Don't trigger extraneous redraws on font / text-attr changes

Any time that these change, we shouldn't need to force a redraw on
the widget either.

widget: Don't queue redraws for properties that don't affect anything

Properties like transition-property might change when hovering over
something, even if the property itself does not change. These properties
don't affect drawing, so don't queue redraws for them.

listbox: Don't queue draws on the entire widget

The set_state_flags code should already queue redraws when needed.

cssimage: Don't start a transition if the start and end are the same

Since a lot of Adwaita sets transition: all, it's easy to end up in a
state where we're making dummy transitions for all of the icons, most of
which we'll never be showing.

Inspector: Preserve selection when rescanning

We are semiregularly repopulating the object tree. Whenever
we do, we loose the selection, needlessly. Do the extra work
to preserve it.

http://bugzilla.gnome.org/show_bug.cgi?id=753130

cleanup: do not set members to null in finalize

cleanup: no need to set members to null when freeing the struct

cleanup: one more g_slist_free_full

Updated POTFILES.skip

Updated Portuguese translation

Updated Portuguese translation

searchenginetracker: Ignore NULL directories

gtkoverlay: Optimize size allocations

Doing raises / lowers here will invalidate the entire child widget
almost every time, since raising each widget causes it to invalidate
that area.

Code cleanup

Use g_slist_free_full more consistently. This commit just converts
the obvious cases where g_slist_forall is directly followed by
g_slist_free.

HC: shade spinbuttons as entries

https://bugzilla.gnome.org/show_bug.cgi?id=746491

gtk-demo: Add a scale example

file chooser: Allow activating without double-click

Interpret a unmodified primary click on the selection like a double
click. This makes it possible to activate a file or open a folder
without using double-click.

gesture single: Use gtk_simulate_touchscreen

Instead of manual tests.

text view: Use gtk_simulate_touchscreen

Instead of repeated manual tests.

entry: Use gtk_simulate_touchscreen

Instead of repeated manual checks.

Add a private function to determine if we simulate

These same two or three tests are currently duplicated
in several places, with separate static variables. Lets
do this in one place.

HC: custom color button fixes

https://bugzilla.gnome.org/show_bug.cgi?id=753051

entry completion: Clarify index in documentation

https://bugzilla.gnome.org/show_bug.cgi?id=579466

testfilechooser: Say Save

Thats a more typical affirmative action in save mode.

Use pango_fc_font_map_config_changed

Use pango_fc_font_map_confit_changed() instead of
pango_fc_font_map_cache_clear().

https://bugzilla.gnome.org/show_bug.cgi?id=748772

GtkMenuButton: explicitly protect against recursion

The visibility toggling happening on ::click() relied implicitly
on the popover animation, but breaks on disabled animations. The
recursion happening within gtk_toggle_button_set_active() (which
triggers ::clicked when changing state) makes this vfunc to run
again, inverting the visibility of the popover in result.

Fix this by explicitly checking about recursion, we want the
button to be toggled to the right state, but we don't want the
callback running again.

https://bugzilla.gnome.org/show_bug.cgi?id=752577

combobox: Refactor cell view treatment

Instead of putting it inside the button manually, put it there properly
by adding a box containing the cell view and the arrow.

Do the same thing in list mode, instead of creating an event box that
tries to behave as a button.

combobox: Handle NULLing the cellview on remove simpler

combobox: Remove unused variables

combobox: Create cell view identically

... in the two places it can be created.

file chooser: Don't crawl recent://

This is unnecessary - all the recent files are in the model already.
It also leads to duplicates, since our duplicate filtering is based
on g_file_equal, which does not consider recent:///blabla with
target-uri=/my/example to be the same as file:///my/example.

widget-factory: Test busy state

Makes it easy to test insensitive state of widgets.i
Also exposes brokenness in the shell busy indicator...

HC: no need for green in scales

HC: fix typo

Expand builder menu documentation

List the supported attributes.

gtk-demo: Move the transparent example under Overlay

gtk-demo: Rename textview demos to "Text View"

Fits better with "Tree View".

gtk-demo: Tweak the markup demo

gtk-demo: Add font features to markup demo

GtkTextView: Support font features

Add a ::font-features attribute to GtkTextTag, and support
font features when inserting Pango markup into a text buffer.

placesview: don't show mount for Computer row

The "Computer" row from places view is an abstract volume
that represents the root of the current partition. As such,
it cannot be mounted or unmounted.

Remove the related item on context menu for Computer row.

fontchooserwidget: use appropriate model for GtkTreeIter

We were previously mixing the model used when filtering with an iter that
has been resolved to the backing model.

This results in both an invalid row index as well as an invalid
iter->stamp.

Fix oversights in templates

Some of our templates were not setting the domain attribute
on the <interface> element.

gtk3-demo: Add a markup demo

HC: make progressbar labels more visible

- use the same colro as regular labels, no
need to be more subtle

https://bugzilla.gnome.org/show_bug.cgi?id=752978

HC: make pushed columns visible

https://bugzilla.gnome.org/show_bug.cgi?id=752977

HC: provide explicit treeview styling

deals with some details like:

- DND stuff
- progressbar
- expanders
- separators

https://bugzilla.gnome.org/show_bug.cgi?id=752975

HC: :checked:hover color fix for menuitems

https://bugzilla.gnome.org/show_bug.cgi?id=752969

HC: make link buttons distinct

https://bugzilla.gnome.org/show_bug.cgi?id=752965

HC: whitespace for sidebar

- make some room around the sidebar icons

https://bugzilla.gnome.org/show_bug.cgi?id=752960

gtk3-demo: Add a scrollbar context menu

Just to demonstrate the capability, not because this is
excellent UI.

gtk3-demo: Add another overlay demo

placesview: simplify code

Since we started adding a persistent Computer item,
all the code related to the view modes became obsolete,
since the view is never empty anymore. So, drop this
dead code and use the plain stack to manage the empty
search results view.

Also, this patch fixed a very annoying keyboard navigation
issue where we couldn't go from the On This Computer to Networks
lists, because they were two separate widgets. Merge the two
lists into a single one, and update headers accordingly.

placesviewrow: hook up finalize

It is needed to hook up the finalize method, since
it's overriden manually.

placessidebar: fix ellipsizing glitch

The places sidebar rows' labels were missing a proper
xalign, which caused the labels to keep shaking while
the sidebar was resized.

Fix that by setting the label's xalign to 0.

placesview: plug some memory leaks

The GFile containing the Computer item was not
properly dereferenced, so plug that leak by both
dereferencing it and adding some reference management
on GtkPlacesViewRow.

placesview: show Computer item

GtkPlacesView widget manages persistent locations,
factoring out GtkPlacesSidebar functionality.

It, however, does not completely shows all sidebar
locations, since Computer is still missing.

Add a Computer item, adjusting some internal behavior
to make that possible.

placesview: don't show network addresses

It is distracting, not relevant and too space
consuming.

Remove the network addresses label.

placesview: fix documentation

The doc for gtk_places_view_get_search_query was being
set twice, while gtk_places_view_set_search_query was
never set.

Fix that by correcting the wrong getter documentation.

placessidebar: fix documentation

Use the same explanation of why GtkPlacesSidebar::show-connect-server
was deprecated and its replacements on getter and setter.

Also, fix a mistakenly deprecated function.

inspector: Show monitor information

This is useful to verify the monitor information returned by
other backends.

inspector: Show wayland display information

Don't call it "X display" when we show useful information under
Wayland as well.

wayland: Return a display name

Make gdk_screen_make_display_name() return a likely correct
name for the Wayland socket we are using.

x11: Make selection handling work across screens

When dealing with selection events, we might see windows from
other screens in the requestor field. The current x11 backend
code fails to wrap these in a foreign GdkWindow, since we
don't have the corresponding GdkScreen anymore. Work around
this by creating such 'foreign screens' on demand. We still
maintain the 1:1 relation between the display and the screen
returned by gdk_display_get_default_screen().

https://bugzilla.gnome.org/show_bug.cgi?id=721398

Update Catalan translation

Plug a memory leak

GtkCssProvider was using the wrong unref function for GtkCssKeyframe
objects. Funnily, this didn't crash at all, the refcount was at the
same place in the struct - it just didn't free all the memory, causing
valgrind to complain.